Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Added instructions for CmdStan tarball builder #202

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bbbales2
Copy link
Member

@bbbales2 bbbales2 commented Feb 22, 2021

Adding instructions to use the CmdStan tarball builder Github action (https://github.com/stan-dev/cmdstan/actions/workflows/tarball.yml)

Here is a build of the initial instructions: cmdstan-tarball-builder.html.zip (Edit: updated)

@bbbales2
Copy link
Member Author

@Dashadower @hyunjimoon give this a go and tell me if these instructions are clear or not.

I'm not super happy with the formatting on the font size/formatting on the screenshots. If you can think of a better way of presenting the information, lemme know. As plain text I think it's sometimes frustrating to figure out exactly where to click.

@Dashadower
Copy link
Member

Thanks! It looks perfectly fine for me, at least on mobile.

@hyunjimoon
Copy link

Yes, I have asked rok and have built the current FASO version here. Nice feature.

Copy link
Member

@rok-cesnovar rok-cesnovar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edits and comments and such.

Formatting and images seem fine to me.

@bbbales2
Copy link
Member Author

@rok-cesnovar I didn't know quite where this document should live so I've just left it as an open pull so I can send people here (the html in the zip file at the top is up to date). I made it a case study cause then I could have images, but I don't think I want to put it on the case studies page like a feature we're gonna support long term or anything.

Next time I look at the new Math contributor docs I'll move it in there. In the interim, let's just leave this pull open.

@bbbales2 bbbales2 changed the title Added instructions for CmdStan tarball builder [WIP] Added instructions for CmdStan tarball builder Feb 26, 2021
@rok-cesnovar
Copy link
Member

Cool, on board with that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants